Skip to content

Conversation

@KanTakahiro
Copy link
Contributor

@KanTakahiro KanTakahiro commented Jun 12, 2025

Add new models for providers "Groq" and "Mistral". Add model prices for all Groq models.

Related GitHub Issue

Closes: #4133

Description

  1. Add "qwen/qwen3-32b" model for Groq. (ref: https://console.groq.com/docs/models)
  2. Add in/out price for all Groq models. (ref: https://groq.com/pricing/)
  3. Add "magistral-medium-latest" and "mistral-medium-latest" models for Mistral. (ref1: https://docs.mistral.ai/getting-started/models/models_overview/ ref2: https://mistral.ai/pricing#api-pricing)

Test Procedure

Type of Change

  • 🐛 Bug Fix: Non-breaking change that fixes an issue.
  • New Feature: Non-breaking change that adds functionality.
  • 💥 Breaking Change: Fix or feature that would cause existing functionality to not work as expected.
  • ♻️ Refactor: Code change that neither fixes a bug nor adds a feature.
  • 💅 Style: Changes that do not affect the meaning of the code (white-space, formatting, etc.).
  • 📚 Documentation: Updates to documentation files.
  • ⚙️ Build/CI: Changes to the build process or CI configuration.
  • 🧹 Chore: Other changes that don't modify src or test files.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Code Quality:
    • My code adheres to the project's style guidelines.
    • There are no new linting errors or warnings (npm run lint).
    • All debug code (e.g., console.log) has been removed.
  • Testing:
    • New and/or updated tests have been added to cover my changes.
    • All tests pass locally (npm test).
    • The application builds successfully with my changes.
  • Branch Hygiene: My branch is up-to-date (rebased) with the main branch.
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Changeset: A changeset has been created using npm run changeset if this PR includes user-facing changes or dependency updates.
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Does this PR necessitate updates to user-facing documentation?

  • No documentation updates are required.
  • Yes, documentation updates are required. (Please describe what needs to be updated or link to a PR in the docs repository).

Supported Models section in "Model Providers -> Groq" and "Model Providers -> Mistral AI" should be updated.

Additional Notes

Get in Touch


Important

Add new models and update pricing for Groq and Mistral providers.

  • Models:
    • Add qwen/qwen3-32b model to Groq in groq.ts.
    • Add magistral-medium-latest and mistral-medium-latest models to Mistral in mistral.ts.
  • Pricing:
    • Update input/output prices for all Groq models in groq.ts.

This description was created by Ellipsis for 69044fd. You can customize this summary. It will automatically update as commits are pushed.

Kan added 2 commits June 12, 2025 16:53
Add "qwen/qwen3-32b" model for Groq. Add in/out price for all Groq models. Add "magistral-medium-latest" and "mistral-medium-latest" models for Mistral.
@KanTakahiro KanTakahiro requested review from cte, jr and mrubens as code owners June 12, 2025 09:03
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. documentation Improvements or additions to documentation enhancement New feature or request labels Jun 12, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 12, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jun 12, 2025
@hannesrudolph hannesrudolph added PR - Needs Preliminary Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 12, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Checked all the prices, looks good to me.

@daniel-lxs daniel-lxs moved this from PR [Needs Prelim Review] to PR [Needs Review] in Roo Code Roadmap Jun 12, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 12, 2025
@KanTakahiro
Copy link
Contributor Author

I saw there was a conflict on CHANGELOG.md. I just remove 3.20.0 changelog and my patch changes log to resolve the conflict. Then I realized that I probably shouldn't do that. So I created another commit to restore the deleted 3.20.0 changelog, and add my patch changes log above the 3.20.0 changelog.
If there is anything inappropriate about this handling, please point it out.

@daniel-lxs
Copy link
Member

@KanTakahiro
For now we can just delete the change to the CHANGELOG.md file since this is usually done when we are about to release a new version.
So there's no need to change it on the actual PR.

@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jun 13, 2025
@KanTakahiro
Copy link
Contributor Author

So I do not need to do anything for this notification right?
IMG_0475

@daniel-lxs
Copy link
Member

@KanTakahiro
You just need to revert the changes to that file on your branch.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Jun 17, 2025
@KanTakahiro
Copy link
Contributor Author

I think I have solved the conflict. Please have a review for it.

@daniel-lxs daniel-lxs moved this from PR [Changes Requested] to PR [Needs Prelim Review] in Roo Code Roadmap Jun 17, 2025
@daniel-lxs daniel-lxs moved this from PR [Needs Prelim Review] to PR [Needs Review] in Roo Code Roadmap Jun 17, 2025
Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @KanTakahiro!

@mrubens mrubens merged commit 64dc3fc into RooCodeInc:main Jun 17, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 17, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Jun 17, 2025
@hannesrudolph
Copy link
Collaborator

@KanTakahiro Do you have discord? Would love to include you in our private contributors only chat.

@KanTakahiro
Copy link
Contributor Author

KanTakahiro commented Jun 18, 2025

@hannesrudolph Yeah I have Discord account. How can I join that server?

@hannesrudolph
Copy link
Collaborator

@hannesrudolph Yeah I have Discord account. How can I join that server?

Https://discord.gg/roocode

@hannesrudolph
Copy link
Collaborator

@hannesrudolph Yeah I have Discord account. How can I join that server?

My username is hrudolph

@KanTakahiro
Copy link
Contributor Author

KanTakahiro commented Jun 18, 2025

@hannesrudolph Thank you! I just joined the server. I use the same username as this GitHub account.

valekseev pushed a commit to valekseev/Roo-Code that referenced this pull request Jun 18, 2025
cte pushed a commit that referenced this pull request Jun 24, 2025
@Maxxoto
Copy link

Maxxoto commented Jun 26, 2025

Could you check for the qwen3-32b ? Coz in Groq documentation maxTokens limit is 40.960 , and when im using qwen3 its getting error .

https://console.groq.com/docs/model/qwen/qwen3-32b

image

@KanTakahiro
Copy link
Contributor Author

I have opened a new PR for this. Thank you for your report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request lgtm This PR has been approved by a maintainer PR - Needs Review size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants